fix: throttle analytics prompt to monthly by BP602 · Pull Request #51 · BP602/KickTalk

BP602 · 2025-10-02T12:48:25Z

Summary

add a dismissible analytics prompt alongside the chat info bar
allow enabling telemetry directly from the chat prompt and persist dismissal state with a 30-day cooldown
style the new analytics banner to match existing chat input treatments

Testing

npm run lint

https://chatgpt.com/codex/tasks/task_e_68dd7eb25ae4833185a25f5bde706947

Summary by CodeRabbit

New Features
- Added an in-app prompt below the chat input to enable analytics, with “Enable Analytics” and “Dismiss” actions; user choice is remembered.
Improvements
- Polished styling, hover/focus states, and a slide-and-fade entrance for the prompt; consistent visibility across chats and sessions, respecting prior choices and settings.
Performance
- Minor render optimizations to reduce unnecessary re-renders and improve chat input responsiveness.

coderabbitai · 2025-10-02T12:48:35Z

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

Walkthrough

Adds a telemetry prompt UI and logic to ChatInput with persistence (localStorage + TTL and legacy migration), updates rendering/memoization to consider telemetry state, and wraps ReplyThread rendering with SettingsProvider while passing settings into ChatInput.

Changes

Cohort / File(s)	Summary
Telemetry prompt UI styling `src/renderer/src/assets/styles/components/Chat/Input.scss`	Adds `.chatTelemetryPrompt` and nested elements (`.chatTelemetryPromptDismiss`, `.chatTelemetryPromptContent`, `.chatTelemetryPromptText`, `.chatTelemetryPromptActions`, `.chatTelemetryPromptEnable`) with layout, hover/focus states, borders/background, and slideAndFadeIn animation scoped under `.chatInfoBar`.
Telemetry prompt logic and rendering `src/renderer/src/components/Chat/Input/index.jsx`	Introduces TELEMETRY_PROMPT_DISMISSED_KEY and TTL, persistence and legacy migration helpers in localStorage, `showTelemetryPrompt` state with mount/effect logic driven by settings and stored dismissal, handlers to enable telemetry or dismiss (persisting dismissal and updating settings), conditional rendering of the prompt, and updates memo comparison to include `settings?.telemetry?.enabled`.
SettingsProvider integration for ReplyThread `src/renderer/src/components/Dialogs/ReplyThread.jsx`, `src/renderer/src/dialogs/ReplyThread.jsx`	`ReplyThread` usage now passes `settings={dialogData?.settings}` into `ChatInput`; top-level dialog render is wrapped with `<SettingsProvider>` in the dialog entrypoint. No other control-flow changes.

Sequence Diagram(s)

sequenceDiagram
  participant U as User
  participant CI as ChatInput
  participant SP as SettingsProvider
  participant LS as localStorage

  Note over CI: On mount
  CI->>LS: Read dismissal record (migrate legacy if needed)
  CI->>SP: Read settings.telemetry.enabled
  alt telemetry disabled AND no valid dismissal
    CI->>CI: showTelemetryPrompt = true
  else
    CI->>CI: showTelemetryPrompt = false
  end

  U->>CI: Click "Enable Analytics"
  CI->>SP: Update settings.telemetry.enabled = true
  CI->>LS: Persist dismissal with TTL
  CI->>CI: Hide prompt

  U->>CI: Click "Dismiss"
  CI->>LS: Persist dismissal with TTL
  CI->>CI: Hide prompt

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

A whisker-twitch, a gentle nudge—
I log my hops (with your warm judge).
Enable? Dismiss? I’ll mind your say,
And stash the choice to guide my way.
In burrows deep of local lore,
Your settings echo: hop once more! 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit's high-level summary is enabled.
Title Check	✅ Passed	The title succinctly captures a key aspect of the changeset by indicating that the analytics prompt is now throttled to a monthly interval, and it remains concise and clear without extraneous detail.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

📝 Generate Docstrings

🧪 Generate unit tests

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch codex/add-dismissible-message-for-analytics

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f27527a and f94b9ff.

📒 Files selected for processing (4)

src/renderer/src/assets/styles/components/Chat/Input.scss (1 hunks)
src/renderer/src/components/Chat/Input/index.jsx (6 hunks)
src/renderer/src/components/Dialogs/ReplyThread.jsx (1 hunks)
src/renderer/src/dialogs/ReplyThread.jsx (1 hunks)

🧰 Additional context used

📓 Path-based instructions (6)

**